Skip to content

test: Fix L0_openai--trtllm#8543

Merged
mc-nv merged 1 commit into
r25.11from
yinggeh/tri-309-2511-l0_openai_trtllm-base
Nov 25, 2025
Merged

test: Fix L0_openai--trtllm#8543
mc-nv merged 1 commit into
r25.11from
yinggeh/tri-309-2511-l0_openai_trtllm-base

Conversation

@yinggeh

@yinggeh yinggeh commented Nov 25, 2025

Copy link
Copy Markdown
Contributor

What does the PR do?

Partially fixed L0_openai--trtllm

FAILED tests/test_tool_calling.py::TestAsyncClientToolCalling::test_tool_call_with_named_tool_choice
FAILED tests/test_tool_calling.py::TestAsyncClientToolCalling::test_tool_call_with_required_tool_choice

Checklist

  • PR title reflects the change and is of format <commit_type>: <Title>
  • Changes are described in the pull request.
  • Related issues are referenced.
  • Populated github labels field
  • Added test plan and verified test passes.
  • Verified that the PR passes existing CI.
  • Verified copyright is correct on all changed files.
  • Added succinct git squash message before merging ref.
  • All template sections are filled out.
  • Optional: Additional screenshots for behavior/output changes with before/after.

Commit Type:

Check the conventional commit type
box here and add the label to the github PR.

  • test

Related PRs:

Where should the reviewer start?

Test plan:

  • CI Pipeline ID:

Caveats:

Background

Related Issues: (use one of the action keywords Closes / Fixes / Resolves / Relates to)

  • closes GitHub issue: #xxx

@yinggeh
yinggeh requested review from pskiran1 and whoisj November 25, 2025 04:07
@yinggeh yinggeh self-assigned this Nov 25, 2025
@yinggeh yinggeh added the PR: test Adding missing tests or correcting existing test label Nov 25, 2025
@mc-nv
mc-nv merged commit ac5001f into r25.11 Nov 25, 2025
3 checks passed
@mc-nv
mc-nv deleted the yinggeh/tri-309-2511-l0_openai_trtllm-base branch November 25, 2025 17:23
mc-nv pushed a commit that referenced this pull request Nov 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

PR: test Adding missing tests or correcting existing test

Development

Successfully merging this pull request may close these issues.

3 participants